Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update DomainEntity_EmailUrlInfo.yaml #11369

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

MSJosh
Copy link

@MSJosh MSJosh commented Oct 31, 2024

Removed materialize as it causes a memory issue on larger data sets. Improved performance of query by splitting the logic.

Required items, please complete

Change(s):
-Removed materialize() functions
-Split single query into two different queries to improve performance
-Cleaned up sub queries

Reason for Change(s):

  • For larger data sets this would lead to memory issues and would fail to run or lead to shorter look back periods.
  • Resolves Issue 11340

Version Updated:

  • 1.0.2

Testing Completed:

  • Yes

Checked that the validations are passing and have addressed any issues that are present:

  • Yes

Removed materialize as it causes a memory issue on larger data sets.  Improved performance of query by splitting the logic.
@MSJosh MSJosh requested review from a team as code owners October 31, 2024 16:20
@MSJosh
Copy link
Author

MSJosh commented Oct 31, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant